Revert fetchDataDeferred behaviour on Widgets page#872
Revert fetchDataDeferred behaviour on Widgets page#872sars wants to merge 30 commits intoerikras:simple-routerfrom
Conversation
`import { Navbar, Nav, NavItem } from 'react-bootstrap';` includes all react-bootstrap modules. By including the modules explicitly (i.e. `import Navbar from 'react-bootstrap/lib/Navbar';`), we can decrease the size of the production js-file and speed up hot reloading drastically.
docs(guide): Add 'Adding Text to Home Page' guide
Update react-bootstrap imports
remote duplicate conf
Fix link to redux-devtools
hoist-non-react-statics no longer needed
There was a problem hiding this comment.
this definitely solves the problem, but it's a patch. For example, we have them everywhere in our app and it'd be too routine and not pretty to have if (__SERVER__) { everywhere. This does show how to solve it though
There was a problem hiding this comment.
It seems for me, that logic of fetchDataDeferred was not really obvious...
Of course, I can add new static method , analogue of fetchDataDeferred, but i'd like to find more elegant solution...
There was a problem hiding this comment.
hmm weird, i thought i replied to this. Sorry for the late reply. I do think a more elegant solution would be best, not sure how other than another static method that will be called after transition or componentDidMount. Or it could even be in a form of a helper? So a callback helper to be called, then youd save having two static methods.
There was a problem hiding this comment.
I thought... may be for this purposes we can simply create new function that will wrap promise and return promise or undefined, depends where we are - on server or on client...
There was a problem hiding this comment.
... depends where we are - on server or on client...
check window object may work in this case
const isServerSide = typeof window !== 'object';
reference PR at redux-async-connect #23
There was a problem hiding this comment.
@sars I really like redial's approach on defer:
https://github.com/markdalgleish/redial
|
Is this linked to an open issues? |
|
Duh.. should have read: #833 |
sars
force-pushed
the
simple-router-data-deferred
branch
from
da8f2c2 to
e60b23c
Compare
redux-simple-router updated to react-router-redux 2.1.0
Better initialization of react router redux on server-side (with original url)
several modules updated
Migration to redux simple router and async-props
feat(ci): add support for circle-ci
docs(guide): Add 'Adding A Page' guide
sars
force-pushed
the
simple-router-data-deferred
branch
from
e60b23c to
b1f8dd1
Compare
10 participants
No description provided.